Skip to content

[#610] SearchView가 뜰 때 키보드 포커싱이 되지 않는 현상을 해결한다#645

Merged
opficdev merged 3 commits into
developfrom
fix/#610-SearchView
Jun 26, 2026
Merged

[#610] SearchView가 뜰 때 키보드 포커싱이 되지 않는 현상을 해결한다#645
opficdev merged 3 commits into
developfrom
fix/#610-SearchView

Conversation

@opficdev

Copy link
Copy Markdown
Owner

🔗 연관된 이슈

🎯 의도

  • SearchView가 표시된 직후 검색 필드가 자동 포커스되지 않는 문제 해결

📝 작업 내용

📌 요약

SearchView 진입 시 검색 상태 변경을 다음 main queue 턴에 수행하도록 조정

🔍 상세

  • 기존 SearchViewModel 시절의 DispatchQueue.main.async 기반 포커스 요청 타이밍 복원
  • TCA 전환 이후 onAppear에서 즉시 isSearching을 변경하던 흐름을 기존 동작과 동일한 형태로 조정
  • SearchView 표시 직후 .searchable이 붙는 시점과 포커스 요청 시점의 경합 완화

📸 영상 / 이미지 (Optional)

@opficdev opficdev self-assigned this Jun 26, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request wraps the state binding update in SearchView.swift's onAppear modifier inside a DispatchQueue.main.async block. The review feedback correctly points out that performing asynchronous dispatch directly from the View violates TCA's unidirectional data flow principles and hinders testability. It is recommended to handle this side effect within the Reducer using an Effect instead.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread Application/DevLogPresentation/Sources/Search/SearchView.swift Outdated
@opficdev opficdev merged commit 088d922 into develop Jun 26, 2026
6 checks passed
@opficdev opficdev deleted the fix/#610-SearchView branch June 26, 2026 04:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SearchView가 뜰 때 키보드 포커싱이 되지 않는 현상을 해결한다

1 participant